Skip to content

ZA2 Tersia completed karma project#362

Open
tersiakoetzee wants to merge 2 commits into
CodeYourFuture:masterfrom
tersiakoetzee:master
Open

ZA2 Tersia completed karma project#362
tersiakoetzee wants to merge 2 commits into
CodeYourFuture:masterfrom
tersiakoetzee:master

Conversation

@tersiakoetzee
Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

Comment thread css/style.css
Comment on lines +78 to +90
.icon-item {
padding: 10px;
margin: 10px;
text-align: center;

}

footer {
flex-direction: row;
flex-wrap: wrap;
flex-direction: column;
text-align: center;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should have formatted the document when you are done, so that your CSS properties can align to improve readability.

Copy link
Copy Markdown

@AnthonyMogotlane AnthonyMogotlane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work, well done!
In future try to have some comments in your work, so we can know which styling is for which section.

@AnthonyMogotlane
Copy link
Copy Markdown

Add some margins between the icons and the header.
footer

Navigation links are not centered with the image.
navigation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants